Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update ATOMIC URLs #71

Merged
merged 3 commits into from
Jul 27, 2021
Merged

Update ATOMIC URLs #71

merged 3 commits into from
Jul 27, 2021

Conversation

xychen-ocn
Copy link

Updated all the NCEI data URLs in the P3 folder.

@RobertPincus RobertPincus added the data source changed Fix for vanished source locations. label Jul 26, 2021
@RobertPincus
Copy link
Contributor

@leifdenby This PR, based on the current version of master, is failing because the radiosonde datasets hosted on derby.eu are not able to be opened. Should we keep pointing to this data, or should we update to the "version of record" e.g. at AERIS? @d70-t your thoughts?

@d70-t
Copy link
Contributor

d70-t commented Jul 27, 2021

There was #26 which is not yet resolved. It ended up in a discussion about where / how to provide backup sources of this dataset.

However, as far as I can see, that referenced issue also mentions that there is a new version of the radiosonde dataset which should be included into the catalog and subsequently the old version should probably be removed anyways. I'll try to check if I can make a PR to include the newer radiosonde datasets. That will probably fix this issue as well.

Copy link
Contributor

@d70-t d70-t left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me.

@d70-t d70-t merged commit 7635aed into eurec4a:master Jul 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data source changed Fix for vanished source locations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants